Skip to content

LCORE-1217: bugfix: missing libtiff openjpeg2 lcms2 libjpeg-turbo libwebp#1034

Merged
tisnik merged 1 commit into
lightspeed-core:mainfrom
raptorsun:missing_lib
Jan 21, 2026
Merged

LCORE-1217: bugfix: missing libtiff openjpeg2 lcms2 libjpeg-turbo libwebp#1034
tisnik merged 1 commit into
lightspeed-core:mainfrom
raptorsun:missing_lib

Conversation

@raptorsun
Copy link
Copy Markdown
Contributor

@raptorsun raptorsun commented Jan 21, 2026 •
edited by coderabbitai Bot
Loading

Description

add missing libraries from RHOAI wheels

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Tools used to create PR

Identify any AI code assistants used in this PR (for transparency and review context)

  • Assisted-by: (e.g., Claude, CodeRabbit, Ollama, etc., N/A if not used)
  • Generated by: (e.g., tool name and version; N/A if not used)

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Summary by CodeRabbit

  • Chores
    • Expanded container image runtime dependencies to include additional libraries for image processing and multimedia support (libtiff, openjpeg2, lcms2, libjpeg-turbo, libwebp) alongside existing utilities.

✏️ Tip: You can customize this high-level summary in your review settings.

@raptorsun
bugfix: missing libtiff openjpeg2 lcms2 libjpeg-turbo libwebp
46da4c5 
Signed-off-by: Haoyu Sun <hasun@redhat.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 21, 2026 •
edited
Loading

Walkthrough

Three configuration files are updated to add five new runtime libraries (libtiff, openjpeg2, lcms2, libjpeg-turbo, libwebp) to the container image build dependencies. The Containerfile and rpms.in.yaml are updated with the new packages, and rpms.lock.yaml is populated with lock entries for these packages across both aarch64 and x86_64 architectures.

Changes

Cohort / File(s) Summary
Container Build Configuration
Containerfile		 Updated microdnf install command to include five additional runtime libraries alongside existing packages
Package Specification
rpms.in.yaml		 Expanded packages list from single-line to multi-line array format, adding libtiff, openjpeg2, lcms2, libjpeg-turbo, and libwebp
Package Lock File
rpms.lock.yaml		 Added lock entries for six packages (jbigkit-libs, lcms2, libjpeg-turbo, libtiff, libwebp, openjpeg2) across aarch64 and x86_64 architectures with metadata (url, repoid, size, checksum, evr, sourcerpm)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • tisnik
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the primary change: adding missing libraries (libtiff, openjpeg2, lcms2, libjpeg-turbo, libwebp) to the container and package configurations, matching all file modifications in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

tisnik
tisnik approved these changes Jan 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@tisnik tisnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tisnik tisnik merged commit 10b8298 into lightspeed-core:main Jan 21, 2026
20 of 22 checks passed
@raptorsun raptorsun deleted the missing_lib branch January 22, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tisnik tisnik tisnik approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@raptorsun @tisnik